Keep indenting text when 'shr-fill-text' is nil (bug#69555)
authorKévin Le Gouguec <kevin.legouguec@gmail.com>
Sun, 3 Mar 2024 16:20:56 +0000 (17:20 +0100)
committerKévin Le Gouguec <kevin.legouguec@gmail.com>
Fri, 15 Mar 2024 06:50:31 +0000 (07:50 +0100)
commit09ab66935154ea0cc4a351b8320bc0e9276b7780
tree332a7b800b606fe7d4caacaea32b4b7fea7b74c1
parent1be33963f068b884d1f8cbd37372638c47a79e84
Keep indenting text when 'shr-fill-text' is nil (bug#69555)

The 'shr-fill-...' functions handle both hard-filling (adding
newlines to break long lines) and indentation.  Setting
'shr-fill-text' to nil currently causes these functions to be
short-circuited completely, so e.g. blockquotes are no longer
indented, whereas the intent of this user option is only to
prevent hard-filling to let visual-line-mode reflow text.

* lisp/net/shr.el (shr-fill-lines): Document that the function
handles more than just filling; move the 'shr-fill-text' check...
(shr-fill-line): ... here, after indentation has been taken care
of.
* test/lisp/net/shr-resources/blockquote.html:
* test/lisp/net/shr-resources/blockquote.txt: New test resources.
* test/lisp/net/shr-tests.el (shr-test--rendering-check): Rename
from 'shr-test', to make the relationship with the 'rendering'
testcase clearer; prefer 'file-name-concat' to 'format'; raise
ERT failure if need be, calling (ert-fail ...) directly instead
of (should (not (list ...))).
(shr-test--rendering-extra-configs): New variable to easily check
that user customizations do not degrade rendering.
(rendering): Consult that new variable; delegate failure-raising
to reduce duplication.
lisp/net/shr.el
test/lisp/net/shr-resources/blockquote.html [new file with mode: 0644]
test/lisp/net/shr-resources/blockquote.txt [new file with mode: 0644]
test/lisp/net/shr-tests.el